-
Notifications
You must be signed in to change notification settings - Fork 38.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
hide operator when describe pod with empty value tolerations #47380
hide operator when describe pod with empty value tolerations #47380
Conversation
/lgtm |
@fabianofranz Can you take a look at this? Need one more approve, thanks. |
/cc @kubernetes/sig-cli-maintainers |
/approve no-issue |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dims, fabianofranz, kevin-wangzefeng Associated issue requirement bypassed by: fabianofranz The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
@davidopp can we have this in 1.7? |
@kevin-wangzefeng technically it should have an issue against it at this stage of the game now. |
Automatic merge from submit-queue |
@timothysc Thanks, I will submit an issue first next time :) |
What this PR does / why we need it:
The tolerations printing in
kubectl descirbe pod
is not correct when toleration.value is empty, this PR is to fix it.Before:
After:
Also updated tests to cover all possible cases of describing pod with tolerations. See changes in of
TestDescribePodTolerations()
indescribe_test.go
Which issue this PR fixes:
Special notes for your reviewer:
Release note: